Skip to content

Assembler: embed elastic-website-search web component on staging#3411

Merged
reakaleek merged 1 commit into
mainfrom
marshy-cyclamen
May 27, 2026
Merged

Assembler: embed elastic-website-search web component on staging#3411
reakaleek merged 1 commit into
mainfrom
marshy-cyclamen

Conversation

@reakaleek
Copy link
Copy Markdown
Member

Why

We want to dogfood the new elastic-website-search web component (chat input mode) on the staging docs site before rolling it out to production.

What

Adds the <elastic-website-search> web component to the assembler build, gated behind a new WEBSITE_SEARCH feature flag. When enabled, a sticky chat input container appears at the bottom of the content column with a short entrance animation.

The script URL is configurable per environment via website_search_url in assembler.yml — staging points at staging-website.elastic.co and local dev points at a local bundle server. Any environment without a website_search_url renders nothing, keeping production untouched.

Test plan

  • Build with --environment staging and confirm the web component script tag and sticky input container appear in the HTML
  • Build with --environment prod and confirm nothing is rendered
  • Locally, run with --environment dev and verify the input mounts and animates in after ~0.6s

Adds the elastic-website-search web component (chat-mode="input") to the
assembler build, gated behind a new WEBSITE_SEARCH feature flag. The script
URL is configurable per environment via website_search_url in assembler.yml,
so staging and local dev can each point at their own bundle. The chat input
mounts into a sticky container at the bottom of the content column with a
short entrance animation.

Co-Authored-By: Claude Sonnet 4.6 (1M context) <noreply@anthropic.com>
@reakaleek reakaleek requested review from a team as code owners May 27, 2026 20:38
@reakaleek reakaleek requested a review from Mpdreamz May 27, 2026 20:38
@reakaleek reakaleek temporarily deployed to integration-tests May 27, 2026 20:38 — with GitHub Actions Inactive
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 27, 2026

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Enterprise

Run ID: 85ff918f-f7ea-4e1f-b9f6-fe63a75fe1af

📥 Commits

Reviewing files that changed from the base of the PR and between 30e4fca and 83a6b63.

📒 Files selected for processing (7)
  • config/assembler.yml
  • src/Elastic.Documentation.Configuration/Assembler/PublishEnvironment.cs
  • src/Elastic.Documentation.Configuration/Builder/FeatureFlags.cs
  • src/Elastic.Documentation.Site/Assets/assembler.css
  • src/Elastic.Documentation.Site/Layout/_Head.cshtml
  • src/Elastic.Markdown/_Layout.cshtml
  • src/services/Elastic.Documentation.Assembler/Building/AssemblerBuilder.cs

📝 Walkthrough

Walkthrough

This PR adds website search functionality by extending the existing configuration and feature flag infrastructure. The YAML configuration file enables the website search feature flag in staging and dev environments and specifies the appropriate script URLs for each. A new WebsiteSearchScriptUrl property is added to PublishEnvironment and FeatureFlags to carry this configuration through the system. The AssemblerBuilder wires the environment configuration to the feature flags during initialization. The UI is conditionally rendered when the feature is enabled: the script is loaded in the page head, a container is created in the layout, the web component is injected and linked to that container, and CSS animations are applied for the entrance effect.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly summarizes the main change: adding the elastic-website-search web component to staging, which aligns with the core objective of the changeset.
Description check ✅ Passed The description is directly related to the changeset, explaining the why, what, and testing strategy for enabling the website search feature.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
✨ Simplify code
  • Create PR with simplified code
  • Commit simplified code in branch marshy-cyclamen

Warning

Review ran into problems

🔥 Problems

Stopped waiting for pipeline failures after 30000ms. One of your pipelines takes longer than our 30000ms fetch window to run, so review may not consider pipeline-failure results for inline comments if any failures occurred after the fetch window. Increase the timeout if you want to wait longer or run a @coderabbit review after the pipeline has finished.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@reakaleek reakaleek merged commit a7cf57b into main May 27, 2026
25 checks passed
@reakaleek reakaleek deleted the marshy-cyclamen branch May 27, 2026 20:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant